Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat(staff page)-switch layout with ccip #425

Merged
merged 6 commits into from
Aug 26, 2023
Merged

feat(staff page)-switch layout with ccip #425

merged 6 commits into from
Aug 26, 2023

Conversation

SivanYeh
Copy link
Collaborator

@SivanYeh SivanYeh commented Jul 18, 2023

Description

Add a custom layout "ccip" to hide footer and header at page staff, switch the layout based on CCIP value

Steps to Test This Pull Request

  1. Type “/2023/en-us/about/staff?ccip=1” in url
  2. Scroll to the top see if header is hidden
  3. Scroll down the bottom see if footer is hidden

Expected behavior

When ccip is true, footer and header are hidden as picture:
image

If false:
image

Related Issue

#413

add a custom-layout and modify page staff
@SivanYeh SivanYeh requested a review from josix July 18, 2023 13:47
@netlify
Copy link

netlify bot commented Jul 18, 2023

Deploy Preview for ephemeral-sable-89c7e0 ready!

Name Link
🔨 Latest commit ac5d112
🔍 Latest deploy log https://app.netlify.com/sites/ephemeral-sable-89c7e0/deploys/64e0932df375ed000852f34a
😎 Deploy Preview https://deploy-preview-425--ephemeral-sable-89c7e0.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@SivanYeh
Copy link
Collaborator Author

Gotta figure out how to switch the layout with token, and which CCIP token meant to be the key.
Almost there!

add if/else to switch layout with ccip in query
@SivanYeh SivanYeh marked this pull request as ready for review July 21, 2023 14:20
@SivanYeh SivanYeh changed the title feat - add custom layout for staff page feat - (staff) - switch layout with ccip Jul 21, 2023
@SivanYeh SivanYeh changed the title feat - (staff) - switch layout with ccip feat(staff page)-switch layout with ccip Jul 21, 2023
pages/about/staff.vue Outdated Show resolved Hide resolved
layouts/ccip.vue Outdated Show resolved Hide resolved
layouts/ccip.vue Outdated Show resolved Hide resolved
Comment on lines +25 to +31
font-size: 20px;
word-spacing: 1px;
-ms-text-size-adjust: 100%;
-webkit-text-size-adjust: 100%;
-moz-osx-font-smoothing: grayscale;
-webkit-font-smoothing: antialiased;
box-sizing: border-box;
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Minor: 如果可以用 tailwind 的語法代換,可以直接用。太複雜就維持現這樣

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

好建議! 不過我的tailwind還不夠純熟. 先維持這樣. 之後再開一個PR改語法

pages/about/staff.vue Outdated Show resolved Hide resolved
Copy link
Contributor

@yungshenglu yungshenglu left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Copy link
Member

@mattwang44 mattwang44 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM
Please modify the PR based on David's comments.

Copy link
Contributor

@yungshenglu yungshenglu left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@josix josix merged commit 8860765 into main Aug 26, 2023
5 checks passed
@josix josix deleted the issue413 branch August 26, 2023 04:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants